Skip to content

feat(ui): export default emoji#110

Merged
renefloor merged 4 commits intomainfrom
feature/export-default-stream-emoji
May 1, 2026
Merged

feat(ui): export default emoji#110
renefloor merged 4 commits intomainfrom
feature/export-default-stream-emoji

Conversation

@renefloor
Copy link
Copy Markdown
Collaborator

@renefloor renefloor commented May 1, 2026

Submit a pull request

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable)

Description of the pull request

This exports the default emoji, so it's easier to customize it.

Summary by CodeRabbit

  • New Features
    • The default emoji component is now publicly available for use in apps.
  • Improvements
    • Emoji rendering now uses the platform’s native emoji font to ensure consistent appearance and sizing across platforms.

@renefloor renefloor requested a review from a team as a code owner May 1, 2026 13:01
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 1, 2026

Warning

Rate limit exceeded

@xsahil03x has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 57 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 377fe8af-fc7b-456b-a0e0-4bba91c59c50

📥 Commits

Reviewing files that changed from the base of the PR and between dc11396 and 93fb0c7.

📒 Files selected for processing (2)
  • packages/stream_core_flutter/CHANGELOG.md
  • packages/stream_core_flutter/lib/src/components/accessories/stream_emoji.dart
📝 Walkthrough

Walkthrough

Barrel export now publicly re-exports DefaultStreamEmoji; the Unicode emoji renderer explicitly sets TextStyle.fontFamily to platform-native emoji fonts (Apple/Segoe/Noto) while preserving fontFamilyFallback and existing fontSize correction logic.

Changes

Cohort / File(s) Summary
Barrel export
packages/stream_core_flutter/lib/src/components.dart
Removed the hide DefaultStreamEmoji directive so DefaultStreamEmoji is publicly re-exported from the components barrel.
Emoji renderer font selection
packages/stream_core_flutter/lib/src/components/accessories/stream_emoji.dart
Unicode emoji renderer now sets TextStyle.fontFamily to the platform-native emoji font (Apple Color Emoji on iOS/macOS when not web, Segoe UI Emoji on Windows, Noto Color Emoji otherwise) and retains fontFamilyFallback; fontSize correction logic unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I tweaked a barrel, gave an emoji its place,
A font was chosen with gentle, hopping grace,
DefaultStreamEmoji now steps into view,
A tiny change — a brighter chew-chew! 🎨🌈

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: exporting DefaultStreamEmoji for easier customization.
Description check ✅ Passed The description covers the CLA requirement, best practices, and testing status, with a brief explanation of the feature purpose, though it lacks detailed testing information.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/export-default-stream-emoji

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 0/1 reviews remaining, refill in 50 minutes and 57 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@xsahil03x xsahil03x changed the title export default emoji feat(ui): export default emoji May 1, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 1, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.23%. Comparing base (c012cfe) to head (93fb0c7).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...r/lib/src/components/accessories/stream_emoji.dart 0.00% 2 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
- Coverage   30.24%   30.23%   -0.01%     
==========================================
  Files         161      161              
  Lines        6230     6232       +2     
==========================================
  Hits         1884     1884              
- Misses       4346     4348       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In
`@packages/stream_core_flutter/lib/src/components/accessories/stream_emoji.dart`:
- Around line 331-335: The platform-based fontFamily switch in stream_emoji.dart
can return 'Segoe UI Emoji' for Windows even on web, which desyncs with the
Noto-specific web scaling; update the switch expression used to set fontFamily
(the variable named fontFamily) so that Windows is only selected when not kIsWeb
(i.e., use .windows when !kIsWeb), forcing web builds to fall through to 'Noto
Color Emoji'; apply the same guard change to the other identical switch/site
referenced (the second fontFamily switch around the other usage) so all web code
paths consistently use Noto Color Emoji.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 51557936-e91f-41ff-a2f8-e20500f284d4

📥 Commits

Reviewing files that changed from the base of the PR and between c088772 and dc11396.

📒 Files selected for processing (1)
  • packages/stream_core_flutter/lib/src/components/accessories/stream_emoji.dart

@renefloor renefloor merged commit 7d73c98 into main May 1, 2026
7 of 8 checks passed
@renefloor renefloor deleted the feature/export-default-stream-emoji branch May 1, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants